.github/workflows/test-ci.yaml (1)

1-41: GitHub Actions CI 설정이 잘 구성되었습니다!

CI 파이프라인이 적절하게 구성되어 있습니다. 여러 Node.js 버전(18~23)에서 테스트를 실행하여 다양한 환경에서의 호환성을 확인하는 전략이 좋습니다.

몇 가지 개선 제안:

1. 의존성 캐싱을 더 효율적으로 활용할 수 있습니다 (현재 캐시 설정은 되어있지만 최적화 가능).
2. 각 실행 단계에 더 명확한 이름을 추가하면 GitHub Actions UI에서 가시성이 향상됩니다.
3. 테스트 결과 및 커버리지 리포트를 저장하는 단계를 추가하는 것을 고려해 보세요.

 name: Test CI

 on:
   workflow_dispatch:
   push:
     branches: ['main']
   pull_request:
     branches:
       - main

 jobs:
   build-and-test:
     runs-on: ubuntu-latest
     strategy:
       matrix:
         node-version: [18, 19, 20, 21, 22, 23]
+      fail-fast: false
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4

       - name: Set up Node.js ${{ matrix.node-version }}
         uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
           cache: 'pnpm'

       - name: Install pnpm
         run: npm install -g pnpm

       - name: Install dependencies
         run: pnpm install --frozen-lockfile
+        timeout-minutes: 5

       - name: Run lint
+        id: lint
         run: pnpm run lint

       - name: Run tests
+        id: test
         run: pnpm run test
+        timeout-minutes: 10

       - name: Run build
+        id: build
         run: pnpm run build
+        timeout-minutes: 5

src/repositories/__test__/post.repository.test.ts (1)

1-146: PostRepository 테스트 케이스가 잘 작성되었습니다!

테스트 스위트가 PostRepository 클래스의 주요 메서드들을 효과적으로 테스트하고 있습니다. PostgreSQL Pool을 모킹하여 실제 데이터베이스 연결 없이 테스트를 수행하는 방식이 적절합니다.

개선 제안:

1. Edge case 테스트 추가: 빈 결과, null 값, 오류 상황 등의 엣지 케이스에 대한 테스트를 더 추가하면 좋겠습니다.
2. findPostByPostId 메서드의 날짜 필터링(start, end 파라미터) 기능에 대한 더 구체적인 테스트가 필요합니다.
3. 각 테스트 케이스가 단순히 결과값만 확인하는 것이 아니라, 실제 SQL 쿼리가 적절하게 구성되는지 확인하는 테스트를 추가하면 좋겠습니다.

특히 아래와 같이 findPostByPostId 테스트를 확장하면 더 견고해질 것입니다:

// findPostByPostId에 대한 추가 테스트 예시
it('날짜 필터링이 있는 경우 올바른 SQL 쿼리를 생성해야 한다', async () => {
  mockPool.query.mockImplementation((query, params) => {
    // 쿼리 및 파라미터 검증
    expect(params.length).toBe(3);
    expect(params[0]).toBe(1); // postId
    expect(params[1]).toBe('2025-03-01'); // start
    expect(params[2]).toBe('2025-03-08'); // end
    expect(query).toContain('ORDER BY pds.date ASC');
    expect(query).toContain('WHERE pds.post_id = $1');
    expect(query).toContain('AND (pds.date AT TIME ZONE');

    return Promise.resolve({
      rows: [],
      rowCount: 0,
      command: '',
      oid: 0,
      fields: [],
    } as QueryResult);
  });

  await repo.findPostByPostId(1, '2025-03-01', '2025-03-08');
  expect(mockPool.query).toHaveBeenCalled();
});

it('결과가 없는 경우 빈 배열을 반환해야 한다', async () => {
  mockPool.query.mockResolvedValue({
    rows: [],
    rowCount: 0,
    command: '',
    oid: 0,
    fields: [],
  } as QueryResult);

  const result = await repo.findPostByPostId(999);
  expect(result).toEqual([]);
});

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

.github/workflows/test-ci.yaml (1)

50-60: 종속성 설치 및 후속 스텝 검토

• 종속성 설치(pnpm install --frozen-lockfile), 린트(pnpm run lint), 테스트(pnpm run test), 빌드(pnpm run build) 단계가 순차적으로 잘 설정되어 있습니다.

또한, YAMLlint 도구에서 지적한 바와 같이 파일 마지막 줄(line 60) 뒤에 개행문자가 없습니다. 파일 끝에 개행 추가 부탁드립니다.

🧰 Tools

🪛 YAMLlint (1.35.1)

[error] 60-60: no new line character at the end of file

(new-line-at-end-of-file)

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro